-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Check if relation is breaking during relation-broken event #50
Conversation
Depends on #49 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great workaround! - will adopt this in the legacy-mysql
relation in the mysql operator as you recommended
5a26300
to
c561313
Compare
Workaround found: https://bugs.launchpad.net/juju/+bug/2025676/comments/1 |
re: https://bugs.launchpad.net/juju/+bug/2025676/comments/3 it looks like no fix planned. @carlcsaposs-canonical are you planning to rework this? |
@taurus-forever Oops, that's outdated Marcelo found a workaround that I updated the PR with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we must keep an eye on a final fix (the bug won't be fixed).
Meanwhile that's a great solution
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. BTW, great description!
Ported from canonical/mysql-router-k8s-operator#104